-
Notifications
You must be signed in to change notification settings - Fork 4.2k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Migrating block editor BlockPatternsList
component
#56210
Conversation
- Removes `__unstableComposite` imports from `@wordpress/components` - Adds private `Composite*` exports from `@wordpress/components` - Refactors `BlockPatternsList` and `BlockPattern` to use updated `Composite` components - Additionally renames list component to be consistent with codebase
Size Change: +58 B (0%) Total Size: 1.7 MB
ℹ️ View Unchanged
|
Flaky tests detected in 8fa80a5. 🔍 Workflow run URL: https://github.com/WordPress/gutenberg/actions/runs/6958437822
|
packages/block-editor/src/components/block-patterns-list/index.js
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM 🚀
Note: this can be difficult using tab, as each sidebar option is a separate tab stop.
Should we open a separate issue to track this?
packages/block-editor/src/components/block-patterns-list/index.js
Outdated
Show resolved
Hide resolved
See also #56442. |
What?
This PR updates
BlockPatternsList
in@wordpress/block-editor
to use the updatedComposite
implementation from #54225.Why?
In #54225, an updated implementation of
Composite
was added to@wordpress/components
. As per #55224, all consumers ofComposite
need to migrate from the old version to the new version.How?
__unstableComposite
imports from@wordpress/components
Composite*
exports from@wordpress/components
BlockPatternsList
andBlockPattern
to use updatedComposite
componentsTesting Instructions
Testing Instructions for Keyboard